Walkthrough

Two device tree source files for ESP32-based Guition devices were updated. The first file added an I2S interface node (i2s0) with ESP32 I2S bindings, configuring port I2S_NUM_0 and three GPIO pins (42, 2, and 41). The second file added a similar I2S interface node with an ESP32 I2S binding, configured with port I2S_NUM_0, status set to disabled, and three different GPIO pins (0, 18, and 17). Both changes include the esp32_i2s.h header file binding definition.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Devices/guition-jc8048w550c/guition,jc8048w550c.dts (1)

35-42: GPIO 17/18 shared with enabled i2c_external — add a comment for maintainers.

The i2s0 node uses GPIO 17 and 18 which are also used by i2c_external (lines 31–32, enabled) and uart1 (lines 56–57, disabled). Since i2s0 is correctly marked status = "disabled", there's no runtime conflict, but a brief comment noting the mutual exclusivity with i2c_external would help future maintainers understand why this node is disabled and what must be toggled to use I2S on this board.

• X
• Mastodon
• Reddit
• LinkedIn